From fad21d8dda7f0a7b4faf8c17499ffab10bed56a9 Mon Sep 17 00:00:00 2001 From: Emmanuele Bassi Date: Thu, 21 Feb 2019 11:23:54 +0000 Subject: [PATCH] Remove aligned allocation checks The tests were added when we thought we had to align memory allocations for structures including a Graphene type in their members. Graphene added alignment annotations for its types, and we never really used the symbols we set after testing for allocations being aligned out of the box with malloc(), and for aligned allocators. --- config.h.meson | 10 ----- meson.build | 116 ------------------------------------------------- 2 files changed, 126 deletions(-) diff --git a/config.h.meson b/config.h.meson index e7cc20eba5..74a90d3df8 100644 --- a/config.h.meson +++ b/config.h.meson @@ -305,13 +305,3 @@ #mesondefine HAVE_PANGOFT #mesondefine ISO_CODES_PREFIX - -#mesondefine MALLOC_IS_ALIGNED16 - -#mesondefine HAVE_POSIX_MEMALIGN - -#mesondefine HAVE_MEMALIGN - -#mesondefine HAVE_ALIGNED_ALLOC - -#mesondefine HAVE__ALIGNED_MALLOC diff --git a/meson.build b/meson.build index b4063c68d4..df35072746 100644 --- a/meson.build +++ b/meson.build @@ -666,128 +666,12 @@ else graphene_has_gcc = graphene_simd.contains('gcc') endif -malloc_is_aligned = false - -if not meson.is_cross_build() or meson.has_exe_wrapper() - malloc_test = cc.run (''' - #include - #include - - #define COUNT 100 - #define is_aligned(POINTER, BYTE_COUNT) \ - (((uintptr_t)(const void *)(POINTER)) % (BYTE_COUNT) == 0) - int - main (int argc, char *argv[]) - { - void **pointers; - int i, a, min_a; - FILE *f; - int wrote; - pointers = malloc (sizeof (void *) * COUNT); - for (i = 0, min_a = 128; i < COUNT; i++, pointers++) - { - *pointers = malloc (sizeof (void *)); - for (a = 1; a <= 128; a = a * 2) - { - if (!is_aligned (*pointers, a)) - { - a = a / 2; - break; - } - } - if (a > 128) - a = 128; - if (a < min_a) - min_a = a; - } - wrote = fprintf (stderr, "%d", min_a); - if (wrote <= 0) - return 1; - return 0; - } - ''') - - if not malloc_test.compiled() or malloc_test.returncode() != 0 - message ('malloc() alignment test failed, assuming unaligned malloc()') - elif malloc_test.stderr().to_int() >= 16 - malloc_is_aligned = true - cdata.set('MALLOC_IS_ALIGNED16', 1) - endif -else - # TODO: more reasinable cross-compiling defaults? - message ('cross-compiling, assuming unaligned malloc()') -endif - -if cc.get_id() == 'gcc' or cc.get_id() == 'clang' - no_builtin_args = cc.get_supported_arguments(['-fno-builtin']) -else - no_builtin_args = [] -endif - -# Check that posix_memalign() is not a builtin -have_posix_memalign = cc.links('''#include - int main (int argc, char ** argv) { - void *p; - return posix_memalign (&p, 16, argc); - } - ''', - args : no_builtin_args, - name : 'posix_memalign() is not a builtin') -if have_posix_memalign - cdata.set('HAVE_POSIX_MEMALIGN', 1) -endif - -# Check that memalign() is not a builtin -have_memalign = cc.links('''#include - int main (int argc, char ** argv) { - return memalign (16, argc); - } - ''', - args : no_builtin_args, - name : 'memalign() is not a builtin') -if have_memalign - cdata.set('HAVE_MEMALIGN', 1) -endif - -# Check that aligned_alloc() is not a builtin -have_aligned_alloc = cc.links('''#include - int main (int argc, char ** argv) { - return aligned_alloc (16, argc); - } - ''', - args : no_builtin_args, - name : 'aligned_alloc() is not a builtin') - if have_aligned_alloc - cdata.set('HAVE_ALIGNED_ALLOC', 1) -endif - -# Check that _aligned_malloc() is not a builtin -have__aligned_malloc = cc.links('''#include - int main (int argc, char ** argv) { - return _aligned_malloc (argc, 16); - } - ''', - args : no_builtin_args, - name : '_aligned_malloc() is not a builtin') -if have__aligned_malloc - cdata.set('HAVE__ALIGNED_MALLOC', 1) -endif - if graphene_has_sse2 or graphene_has_gcc message('Need aligned memory due to the use of SSE2 or GCC vector instructions') if os_win32 and cc.get_id() == 'gcc' add_global_arguments(['-mstackrealign'], language: 'c') endif - - if (not malloc_is_aligned and - not have_posix_memalign and - not have_memalign and - not have_aligned_alloc and - not have__aligned_malloc) - error('Failed to find any means of allocating aligned memory') - endif - endif subdir('gdk') -- 2.30.2